Skip to content
This repository has been archived by the owner on Aug 26, 2024. It is now read-only.

Seeding Report: Test Tray Seeding Summary Table #34 #173

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

theahegde
Copy link

Pull Request Description

addresses issue #34


Licensing Certification

FarmData2 is a Free Cultural Work and all accepted contributions are licensed as described in the LICENSE.md file. This requires that the contributor holds the rights to do so. By submitting this pull request I certify that I satisfy the terms of the Developer Certificate of Origin for its contents.

@anlai46
Copy link

anlai46 commented May 3, 2024

Overall our team's code successfully checks whether the summary table appears when it's supposed to. Is there any part of the code that can be refactored or improved in any way to make it more maintainable? Also, I think it's still possible that the test might not handle everything because there is possibilities of network or server issues occurring that mess up the testing process. So in situations like that, how can we prepare for those unexpected errors? -Team Anaf

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants